Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow(dead_code) #5157

Closed
wants to merge 1 commit into from
Closed

allow(dead_code) #5157

wants to merge 1 commit into from

Conversation

rustbasic
Copy link
Contributor

@rustbasic rustbasic commented Sep 24, 2024

Copy link

Preview available at https://egui-pr-preview.github.io/pr/5157-patch129
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@lucasmerlin
Copy link
Collaborator

Why is this needed? Are you getting a lint error?

If DummyStorage isn't exported and isn't used anywhere, maybe we can just remove it?

@bircni
Copy link
Contributor

bircni commented Sep 24, 2024

@lucasmerlin I also already wondered but maybe there is something we miss...
I don't get a linting error...

@rustbasic
Copy link
Contributor Author

This is required for the latest version of the Rust compiler.

@emilk emilk closed this Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Latest egui file_dialog compiles with warning
4 participants